modelbutton: Use a builtin icon for the arrow

No need to do this manually.

expander: Fix drawing in rtl

GtkBuiltinIcon leaves the cairo context in unclean state
when drawing, work around that.

Fix a case a height/width confusion

expander: Use a builtin icon

This saves quite a bit of code.

Updated Slovak translation

Updated Portuguese translation

Updated Portuguese translation

label: Allocate gadget before using it

Move the gtk_css_gadget_allocate call before the
gtk_label_update_layout_width call. This fixes the
statusbar label in widget-factory page 2 coming
up fully ellipsized.

notebook: Sync menu labels with page list in visibile_cb

It's where it changes. Size requests have nothing to do with that.

notebook: Don't allocate during size requests

wat?

label: Don't use _gtk_mis_get_padding_and_border()

The function takes into account CSS padding and border, which these days
is handled by the gadget code. Use gtk_misc_get_padding() instead.

imagedefinition: The storage type is not a boolean

Simplify actionbar

We don't need a frame widget here, just render a top border on the
box.

Adwaita: fix action bar styling

Make action bar buttons have some padding again.

icon helper: suppress style_changed when transient

Transient nodes should not propagate style-changed signals
that can cause widgets to get reallocated. This was causing
treeviews and iconviews with pixbuf cells to be constantly
resized and redrawn.

Add more icon entry tests

This test shows that changing icon types works.

entry: Don't clear when setting icons

Clearing the icon doesn't appear to be necessary with
todays code, and it has the unfortunate side-effect of
temoorarily hiding the icon's window, which breaks grabs
and makes us miss the button release event when the icon
is changed from a button press handler.

css node: Use g_set_object

Replace unref + assignment + ref combinations by g_set_object_calls.

entry: Remove some dead code

These defines are no longer used.

Add a way to highlight resizing widgets

https://bugzilla.gnome.org/show_bug.cgi?id=759586

GtkWidget: Don't mix code and declarations

expander: Deprecate style properties

This can all be done with CSS properties now.

expander: Let gadget render focus

This lets us drop lots of allocation fiddling, and reduces
the code that is consulting the style properties we want
to get rid of.

box: Add a way to get the gadget

This will be used in GtkButtonBox.

buttonbox: Convert to gadgets

Set a css name and use a gadget for button box allocation and
drawing.

Fix a problem with entry gadgets

place_windows is accessing the gadgets allocation, so make sure
to allocate the gadget before calling it.

Updated Spanish translation

gdkborder: change types from gint16 to gint

gtkborder: restore original struct definition

We're going to add back the original struct definition removed by
a6e4de28, because using the typedef breaks all sorts of things like
gtkmm and WebKit, and having separate struct definitions allows us to
change the types in GdkBorder from gint16s to gints without breaking
ABI.

Win32: call dnd exit when finalizing the display

Since we init dnd when opening the display, call the corresponding
exit function (which is unused at the moment) when finalizing the
display.

Win32: move CoInitialize to dnd init

Functions requiring CoInitialize are called just in two places:
 - the filechooser thread which calls its own CoInitializeEx
 - the dnd code

Moving CoInitialize in the dnd specific init is cleaner and
we can pair it with the corresponding CoUninitialize since
CoUninitialize should be called as many times as CoInitialize.
Note that it is ok to call this function multiple times, so it
will not break if another codepath will need it in the future.

The patch also replaces the deprecated CoInitialize with the
equivalent call to CoInitializeEx (already used in the filechooser).

GtkSeparator: Don't leak the gadget

entry: use a gadget

Use a gadget for the entry. Selection and progress still
just use a CSS node. Most things seem to work correctly
with non-zero margin or padding.

inspector: Ensure clip is always shown

inspector: Always show clip

It is less confusing than hiding it sometimes.

gadget: Don't warn about unset clip

Instead of warning about an unset clip, simply ignore
empty content clips.

cellview: Convert to gadgets

gadget: Avoid warning for empty containers

Empty boxes end up setting the clip to { 0, 0, 0, 0}, so warning
for a width or height of 0 triggers false positives. Instead,
initialize the clip to clearly invalid values.

gadget: Add more information to warnings

We have the gadget at hand, so we should print out some
information that helps tracking down where it comes from.

gadget: Update docs

Make it clear that out_clip must be set.

gadget: Warn if clip is not set

This leads to hard-to-track-down drawing errors, so better
warn explicitly about it.

frame: Fix gadget conversion

The GtkCssAllocateFunc must set the clip to meaningful values.

frame: Convert to gadgets

As part of this conversion, remove the hardcoded padding around
the label.

Unfortunately, we cannot use the main gadget for drawing the frame
decoration, since we want to draw a custom border instead of the
stock css border that gadgets insist on drawing for us. Therefore,
add an extra gadget with name border and use it just for rendering
the frame.

GtkPopover: Ensure we unset modality before unparenting the popover

Otherwise the gtk_grab_remove() calls on widget destruction will happen
on the default window group, which may leave the real window group
of the popover with a dangling pointer if it is not the default one.

This could be seen on the inspector, open a popover in the properties
list and close the window with alt-F4.

label: Position the text properly

The PangoLayout needs to be positioned according to the content
allocation of the gadget, not the widget's allocation.

cssnode: Redo first/last-child change tracking

Invisible nodes don't change the first/last-child status of the nodes
after/before them. That means we don't have to just check the state of
the adjacent node when modifying this state, but all their siblings
until we hit a visible node.

The same way, a node is not the first child if it has no previous
sibling, it is the first child if it has no previous visible sibling.
This is important for caching in the global lookup cache.

checkbutton: Use a builtin icon

Also, add support to uiltin icons to look up the default size from a
style property.

switch: Deprecate style properties

CSS min-width and min-height on the slider node fit this
perfectly. We still fall back to the slider-width and
slider-height style properties if the CSS properties are
not set.

Simplify some code

Deal with the min-width/min-height fallback the same way
as in GtkBuiltinIcon.

inspector: Use seat grabs for the "inspect widget" feature

And remove some usages of GdkDeviceManager API.

GtkTreeView: Add ignore deprecation statements

We still need access to floating devices at places.

GtkWindow: Add ignore deprecation statements

We still need access to floating devices at places

GtkWindow: Avoid GdkDeviceManager API

We can iterate over the seats' pointers, instead of over master pointers.

GtkWidget: Add some ignore deprecation statements

We still need access to floating devices here.

GtkMenu: Avoid GdkDeviceManager API

We used to pick a master pointer "at random", use the default seat pointer
instead.

GtkComboBox: Avoid GdkDeviceManager API

We used to pick a master pointer "at random", use the default seat pointer
instead.

scrolledwindow: Remove unneeded code

We no longer take a grab here, no need to undo it on grab_notify

x11: Make fallback window dragging use seat grabs

Also, gdk_seat_ungrab() will undo the implicit grab when EWMH dragging
is used.

x11: Add some ignore deprecation statements

In those places we still need access to floating devices, or we need to
use the GdkDeviceManager as its GdkEventTranslator interface.

x11: Avoid gdk_device_manager_get_client_pointer()

We can use the pointer of the default seat instead.

wayland: Avoid GdkDeviceManager APIs

In most places, we can do with the pointer/keyboard of the default seat
instead of the client pointer. We can also remove some code from
gdk_input_init() because we know for sure there's no floating devices to
care about here.

broadway: Add a few ignore deprecations statements around GdkDeviceManager

There's places where we still need to deal with floating devices, which are
unseen by seats. Ignore deprecations and keep using GdkDeviceManager until
we can forget about floating devices.

GdkDisplay: Add a few ignore deprecations statements around GdkDeviceManager

There's places where we still need to deal with floating devices, which are
unseen by seats. Ignore deprecations and keep using GdkDeviceManager until
we can forget about floating devices.

GdkWindow: Add a few ignore deprecations statements around GdkDeviceManager

There's places where we still need to deal with floating devices, which are
unseen by seats. Ignore deprecations and keep using GdkDeviceManager until
we can forget about floating devices.

GdkDisplay: Iterate through seats' pointers/keyboards in older grab functions

Instead of using the GdkDeviceManager.

GdkWindow: Iterate through seats' pointers/keyboards in older grab functions

Instead of using the GdkDeviceManager.

gdkevents: Fallback to the event device seat

There's places where we don't set a seat yet, plus the places
outside GTK+ where events are created, we should warn and fall
back to the master device seat with these.

gdkevents: Ensure the seat data is copied in events

gdk: Set seat on events generated in common code

border: silence a GIScanner warning

gadget: fix copy/paste error in documentation

gdkevents: fix a compiler warning

check button: Deprecate indicator-size

CSS min-width/min-height can do just the same. We still fall
back to the indicator-size style property if the CSS properties
are not set.

check button: Deprecate and ignore ::indicator-spacing

This can be done just as well with CSS margins, so
do that instead for Adwaita.

model button: Document iconic layout

Document how the CSS nodes look if iconic is TRUE. Add
a .model style class for this case and use it in Adwaita.

Revise CSS node documentation

Clarify the use of brackets in the CSS node diagrams:
[] means optional nodes or classes, <> means child widgets.

widget-factory: Make things compile again

1e1064398c7607d197c563c42f8901ac67982a9f broke the build.

When I run make, I should make sure to run it in the right directory.
And not in the gtk/ subdirectory that will never build widget-factory...

widget-factory: Disconnect clipboard watcher on close

The clipboard emit events after the button we connected it to was
already destroyed (on application close for example), so make sure we
don't try to use that dead button.

wayland: Check transient loop

Gdk Wayland backend walks up the transient windows tree, but does not
check for cycles when doing so.

As a result, if two or more windows are transient to each other, the
Wayland gdk backend will enter an infinite loop.

While this is clearly a bug in the application, gtk+/gdk should be more
robust and handle such errors more gracefully.

To avoid looping infinitely at various point in the code, check for a
possible loop when setting the transient relationship and deny the
request to set a window transient for another if that would create a
loop.

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=759299

Fix the fix to the last commit

I should not push while compilation is still going on

Fix previous commit

The scale parameter was missing.

gdk_pixbuf_get_from_window: honor device scale

gdk_pixbuf_get_from_window() paints the given window onto a new cairo
surface. Create that new surface with the same device scale as the
window so that the result is not scaled down on hidpi screens.

This is similar to 657a43e (which was reverted), but doesn't modify the
behavior of gdk_pixbuf_get_from_surface().

https://bugzilla.gnome.org/show_bug.cgi?id=757147

shortcuts: Make sure that hidden shortcuts stay hidden

When doing a gtk_widget_show_all() on the shortcuts window,
accelerators for both RTL and LTR directions are being shown.

Make sure that no-show-all is set by default on hidden shortcuts, and
updated if the widget direction changes.

https://bugzilla.gnome.org/show_bug.cgi?id=759541

3.19.5

paned: Initialize clip properly

This was causing problems in the case when only one of the paned
children is visible - we would use uninitialized memory, leading
to invalide clip regions. Concretely, the signal tab in the inspector
would sometimes not render at all.

renderborder: Actually draw the path as one element

Avoids spurious lines that can sometimes be seen going towards the
center of the widget.

builtinicon: Change the way size is handled

Make min-width/height have preference over the set default size. This
allows shrinking the widget. The default size is only used if min-width
is not set (or explicitly set to 0.

gadget: Add builtin icon gadget

This is to be used in all the places where we now call
gtk_render_activity()/option()/check() etc that in turn call the icon
render function.

rendericon: Move a function

The function is identical if used for builtin icons and regular icons
(as per iconhelper). So split it out in a way that doesn't assume
either.

paned: Allocate the handle input area properly

Drop the margin misuse and use the border allocation of the
handle gadget. We use negative margins to make the border allocation
larger without pushing the paned children out.

gadget: Add api to get the border allocation

This can be useful for giving input windows the right size.

paned: Fix size allocation

We were passing negative widths to height-for-width calls that
didn't expect this.

progressbar: Change the way the progress gadget gets size

Size of the progress element now grows also when it's close to 0 size.

Previously the size was clamped to the minimum size, now it starts
growing from the minimum size.
So for a 100px trough with a 10px min size progress, the sizes of the
progress element change like this:
        old     new
0%      10      10
5%      10      14
10%     10      19
20%     20      28
50%     50      55
100%    100     100

cssimage: Handle 0x0 images

0x0 images can happen when we fail to load an image - or when we
successfully load a 0x0 image.

stylecontext: Builtin images can be (and are) transformed

So don't special-case them out.

css: Move enum

I want to use it for a gadget, so it's better in the generic file.

Really undo the deprecation of separator style properties

I hit the wrong one in the previous commit. For good measure,
undo depecation for all separator-related style properties.
They are still in use.

shortcuts: Fix the circular stack switcher

The buttons need to set min-width now, to make this work.

Undo deprecation of the wide-separators style property

This was premature, as this style property is still used
by GtkTreeView and GtkMenu.

Updated POTFILES.skip